Skip to content

Bump PID used for checkpoint to be > 1000#449

Merged
tjwatson merged 1 commit intoOpenLiberty:vNextfrom
tjwatson:elevateCheckpointPID
Sep 8, 2023
Merged

Bump PID used for checkpoint to be > 1000#449
tjwatson merged 1 commit intoOpenLiberty:vNextfrom
tjwatson:elevateCheckpointPID

Conversation

@tjwatson
Copy link
Copy Markdown
Member

@tjwatson tjwatson commented Sep 8, 2023

No description provided.

Some environments cause higher PIDs to be used in
the container than 100 which leads to potential
conflicts in the PID needed to restore.

This change increases the checkpoint PID to be > 1000.
More investigation is needed to do this efficiently.
If a more efficient way is found then we may want to
increase this to a higher PID value.  To avoid long
delays during a checkpoint the value of 1000 is being
used for now.
@tjwatson tjwatson requested a review from mbroz2 September 8, 2023 17:23
@tjwatson
Copy link
Copy Markdown
Member Author

tjwatson commented Sep 8, 2023

Fixes #448

@tjwatson tjwatson requested a review from leochr September 8, 2023 18:52
Copy link
Copy Markdown
Member

@leochr leochr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks

@tjwatson tjwatson merged commit 0b428b1 into OpenLiberty:vNext Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants